Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert showing wasm transactions in EVM rpc #1861

Merged
merged 1 commit into from
Sep 17, 2024
Merged

Conversation

codchen
Copy link
Collaborator

@codchen codchen commented Sep 17, 2024

Describe your changes and provide context

This reverts https://github.com/sei-protocol/sei-chain/pull/1832/files#diff-905971bab6ab091b4980166748c8ef8aea28bd1d258321badd04c5d9644035c9 and fixes #1859

Testing performed to validate your change

Copy link

codecov bot commented Sep 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.67%. Comparing base (e8e4b3b) to head (b8e921b).
Report is 166 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1861      +/-   ##
==========================================
- Coverage   61.64%   60.67%   -0.98%     
==========================================
  Files         365      260     -105     
  Lines       26178    22849    -3329     
==========================================
- Hits        16138    13864    -2274     
+ Misses       8967     8005     -962     
+ Partials     1073      980      -93     
Files with missing lines Coverage Δ
evmrpc/block.go 68.13% <ø> (+0.10%) ⬆️

... and 193 files with indirect coverage changes

@codchen codchen merged commit dcb6b14 into main Sep 17, 2024
49 checks passed
@codchen codchen deleted the revert-wasm-event-in-evm branch September 17, 2024 13:26
blindchaser added a commit that referenced this pull request Sep 19, 2024
* Release oom hotfix

* Revert showing wasm transactions in EVM rpc (#1861)

* fix: add V580Params specific functions

* chore: update variable name ParamsPreV580

---------

Co-authored-by: yzang2019 <[email protected]>
Co-authored-by: codchen <[email protected]>
blindchaser added a commit that referenced this pull request Sep 24, 2024
* Release oom hotfix

* Revert showing wasm transactions in EVM rpc (#1861)

* fix proto order for deliver_tx_hook_wasm_gas_limit

* add log debug

* Add log

* Fix log

* remove synthetic events from eth_ endpoints & fix param reading

* remove GetParam fallback mechanism & always use GetParamsIfExists

* revert block.go to v.5.8.0-hot-fix-2

* lint fix

---------

Co-authored-by: yzang2019 <[email protected]>
Co-authored-by: codchen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] eth_getBlockByNumber returns txhash with null value
2 participants